Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: check session.Context().Done() in examples/consumergroup #2240

Merged
merged 1 commit into from
Jun 6, 2022
Merged

fix: check session.Context().Done() in examples/consumergroup #2240

merged 1 commit into from
Jun 6, 2022

Conversation

zxc111
Copy link
Contributor

@zxc111 zxc111 commented Jun 5, 2022

I refer the consumeGroupExample, and I meet the same problem with this issue #1192.
So I add the solution to the example.

@dnwe dnwe changed the title fix examples/consumergroup fix: check session.Context().Done() in examples/consumergroup Jun 6, 2022
@dnwe dnwe added the fix label Jun 6, 2022
@dnwe
Copy link
Collaborator

dnwe commented Jun 6, 2022

@zxc111 thanks!

@dnwe dnwe merged commit 0151486 into IBM:main Jun 6, 2022
@zxc111 zxc111 deleted the fix_example branch June 6, 2022 10:37
@fangxlmr fangxlmr mentioned this pull request Oct 9, 2022
Closed
@jonchammer jonchammer mentioned this pull request Oct 31, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dnwe dnwe dnwe approved these changes

Assignees
No one assigned
Labels
fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zxc111 @dnwe